Skip to content

Speed up the docs build checks - #289

Merged
philljj merged 2 commits into
wolfSSL:masterfrom
ejohnstown:ci-pandoc-container
Oct 26, 2026
Merged

philljj merged 2 commits into
wolfSSL:masterfrom
ejohnstown:ci-pandoc-container

Conversation

@ejohnstown

@ejohnstown ejohnstown commented Oct 23, 2026 •

Copy link
Copy Markdown
Contributor

Both build checks spent much of their time installing TeX Live and building doxybook2. They now start from pandoc/latex:latest-ubuntu with only the extra TeX packages the manuals need.

  • Docs Build Test: use the doxybook2 release binary; drop the broken Japanese font download
  • Docker Docs Build Test: build doxybook2 in a parallel stage and run the manual builds with make -j

The build check now runs in pandoc/latex:latest-ubuntu and adds only
the few TeX packages the manual headers use, instead of installing
texlive-full from apt and building doxybook2 through vcpkg.

- install the doxybook2 v1.5.0 release binary
- add titlesec, tocloft, fvextra, listings and fancyhdr with tlmgr
- drop the Japanese font download, which fetched HTML pages
- move actions/checkout to v4 and drop the PATH exports
Copilot AI balanced review requested due to automatic review settings October 23, 2026 21:08

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.

The Docker build now starts from pandoc/latex:latest-ubuntu and adds
only the TeX packages the manual headers use. CI runs the per-manual
docker builds in parallel.

- build the dgarske doxybook2 fork in its own stage so it runs
  alongside the builder setup
- move actions/checkout to v4
@ejohnstown ejohnstown changed the title Run the docs build check in a pandoc container Speed up the docs build checks Oct 23, 2026
@ejohnstown
ejohnstown requested a review from philljj October 23, 2026 22:01
@philljj philljj self-assigned this Oct 26, 2026
@philljj

philljj commented Oct 26, 2026

Copy link
Copy Markdown
Contributor

Thank you, looks like an incredible speedup potentially

@philljj
philljj merged commit 8f4db53 into wolfSSL:master Oct 26, 2026
3 checks passed
@ejohnstown
ejohnstown deleted the ci-pandoc-container branch October 26, 2026 20:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants